-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Steps quiz - fix content and accessibility issue #2399
Conversation
58f8752
to
2e332ff
Compare
2e332ff
to
82c16e0
Compare
Pre-approve upon addition of accessibility report and méli-mélo checklist review. |
82c16e0
to
d142765
Compare
d142765
to
2203800
Compare
@Garneauma changes and additions have been made and pushed to PR. Please review and let me know if it should require any additional changes. |
2203800
to
f63b938
Compare
@Ricokola Could you please translate the working example page so we can test the French version? |
f63b938
to
b441bb2
Compare
b441bb2
to
717a1dc
Compare
@Garneauma Changes identified by @duboisp has been made. Note that I have added a |
717a1dc
to
c77cb7b
Compare
c77cb7b
to
fd5a329
Compare
fd5a329
to
6b93720
Compare
Description:
<progess>
must be accompanied or wrapped by a<label>
element : https://developer.mozilla.org/en-US/docs/Web/HTML/Element/progress#accessibility.